Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

feat(graph-service): add entity node handling and update Docker configurations #100

Merged
merged 3 commits into from
Sep 10, 2024
Merged

feat(graph-service): add entity node handling and update Docker configurations #100

merged 3 commits into from
Sep 10, 2024

Conversation

paul-paliychuk
Copy link
Collaborator

@paul-paliychuk paul-paliychuk commented Sep 10, 2024 •
edited by ellipsis-dev bot
Loading

🚀 This description was created by Ellipsis for commit a73f944

Summary:

This PR updates the graph service by adding entity node handling, modifying DTOs, and updating Docker configurations.

Key points:

  • Dockerfile: Added ENV PORT=8000 and modified CMD to use environment variable PORT.
  • docker-compose.yml: Changed graph service to use zepai/graphiti:latest image and added PORT=8000 to environment variables.
  • server/graph_service/dto/init.py: Added AddEntityNodeRequest to imports and __all__.
  • server/graph_service/dto/common.py: Added uuid field to Message class.
  • server/graph_service/dto/ingest.py: Added AddEntityNodeRequest class.
  • server/graph_service/dto/retrieve.py: Removed search_type from SearchQuery and added center_node_uuid to GetMemoryRequest.
  • server/graph_service/routers/ingest.py: Added /entity-node endpoint to handle AddEntityNodeRequest.
  • server/graph_service/routers/retrieve.py: Added /episodes/{group_id} endpoint to retrieve episodes.
  • server/graph_service/zep_graphiti.py: Added save_entity_node method to ZepGraphiti class.

Generated with ❤️ by ellipsis.dev

@ellipsis-dev ellipsis-dev bot changed the title ... feat(graph-service): add entity node handling and update Docker configurations Sep 10, 2024
ellipsis-dev[bot]
ellipsis-dev bot reviewed Sep 10, 2024
View reviewed changes
Copy link
Contributor

@ellipsis-dev ellipsis-dev bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍 Looks good to me! Reviewed everything up to a73f944 in 24 seconds

More details
  • Looked at 252 lines of code in 10 files
  • Skipped 0 files when reviewing.
  • Skipped posting 1 drafted comments based on config settings.
1. Dockerfile:43
  • Draft comment:
    Add a newline at the end of the file for better formatting and to adhere to best practices.
  • Reason this comment was not posted:
    Confidence changes required: 10%
    The Dockerfile has a missing newline at the end of the file, which is a best practice to include.

Workflow ID: wflow_JK1fRkjLo9waBMbz

You can customize Ellipsis with 👍 / 👎 feedback, review rules, user-specific overrides, quiet mode, and more.

@paul-paliychuk paul-paliychuk merged commit ad2962c into main Sep 10, 2024
6 checks passed
@paul-paliychuk paul-paliychuk deleted the user-scope-search branch September 10, 2024 15:00
@github-actions github-actions bot locked and limited conversation to collaborators Sep 10, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@ellipsis-dev ellipsis.dev ellipsis-dev left review comments

@prasmussen15 prasmussen15 prasmussen15 approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@paul-paliychuk @prasmussen15